Skip to content
This repository has been archived by the owner on Aug 18, 2021. It is now read-only.

Add interface for word related functions #57

Open
wants to merge 10 commits into
base: master
Choose a base branch
from

Conversation

mariatsalakou
Copy link

Added functions related to words described in https://libsemigroups.readthedocs.io/en/latest/words.html

@@ -107,6 +107,9 @@ def cppyy_version():
cppyy.cppdef("#define HPCOMBI_CONSTEXPR_FUN_ARGS")

cppyy.include("libsemigroups/libsemigroups.hpp")
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think you can fix this by doing:

if compare_version_numbers(libsemigroups_version(), "1.1.0") and not compare_version_numbers("1.3.0", libsemigroups_version()):

@@ -126,6 +129,7 @@ def cppyy_version():
from libsemigroups_cppyy.schreier_sims import SchreierSims
from libsemigroups_cppyy.transf import *
from libsemigroups_cppyy.todd_coxeter import ToddCoxeter
from libsemigroups_cppyy.word import number_of_words, cbegin_wilo, cend_wilo, wilo, cbegin_wislo, cend_wislo, wislo, cbegin_sislo, cend_sislo, sislo, cbegin_silo, cend_silo, silo
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

also put these lines inside an "if"

@james-d-mitchell
Copy link
Member

Please add some tests.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants